Content.Shared/_Art/CVars/ArtCVars.cs-17-21 (1)

17-21: ⚠️ Potential issue | 🟡 Minor

Неверный XML-комментарий (копипаст).

Комментарий "URL of the TTS server API" не соответствует переменной TTSClientEnabled.

📝 Предлагаемое исправление

 /// <summary>
-/// URL of the TTS server API.
+/// Whether TTS is enabled on the client.
 /// </summary>
 public static readonly CVarDef<bool> TTSClientEnabled =

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Shared/_Art/CVars/ArtCVars.cs` around lines 17 - 21, Комментарий XML
над объявлением public static readonly CVarDef<bool> TTSClientEnabled
некорректен (копипаст — "URL of the TTS server API"); замените его на корректное
описание состояния этой булевой переменной, например кратко поясняющее, что
переменная включает/отключает TTS-клиент (например: "Enable/disable the TTS
client" или на русском "Включает/выключает TTS-клиент"), сохранив формат
<summary>... </summary> над TTSClientEnabled и не меняя саму сигнатуру
CVarDef<bool>.

Resources/_Art/TTS/tts-voices.yml-93-112 (1)

93-112: ⚠️ Potential issue | 🟡 Minor

Уточнение о статусе голосов в TTS-системе.

Файл содержит идентификаторы синтетических TTS-голосов, а не записанные голоса реальных людей. TTS-система генерирует речь из текста, а не воспроизводит подготовленные аудиофайлы. Хотя включение имён реальных политиков (Байден, Обама, Трамп) среди голосов потенциально может создать правовые вопросы относительно прав на персональность и использование имён известных личностей, контекст некоммерческого проекта (согласно LICENSE.TXT) и игровая среда значительно снижают этот риск. Рекомендуется добавить комментарий в код для уточнения парody/развлекательного характера этих голосовых идентификаторов.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Resources/_Art/TTS/tts-voices.yml` around lines 93 - 112, The YAML currently
lists synthetic TTS voice identifiers for "Джо Байден" (speaker/id: biden),
"Барак Обама" (speaker/id: obama) and "Дональд Трамп" (speaker/id: trump)
without clarifying they are synthetic/parody; add a brief comment near these
entries (or at the top of Resources/_Art/TTS/tts-voices.yml) stating that these
are synthetic TTS voices generated from text (not recorded audio), intended for
parody/entertainment in a non-commercial/game context and not representations of
real recorded performances, to reduce legal ambiguity and clarify intent.

Content.Client/Options/UI/Tabs/AudioTab.xaml-11-11 (1)

11-11: ⚠️ Potential issue | 🟡 Minor

Нарушение паттерна локализации.

Строки "Громкость TTS:" и "Включить ТТС" жёстко закодированы на русском языке, тогда как остальные элементы используют {Loc '...'} для поддержки локализации. Это нарушает консистентность и затрудняет перевод интерфейса.

🌐 Предлагаемое исправление

-<ui:OptionSlider Name="SliderVolumeTts" Title="Громкость TTS:"/> <!-- Art-TTS  -->
+<ui:OptionSlider Name="SliderVolumeTts" Title="{Loc 'ui-options-tts-volume'}"/> <!-- Art-TTS  -->

-<CheckBox Name="TtsClientCheckBox" Text="Включить ТТС"/> <!-- Art-TTS -->
+<CheckBox Name="TtsClientCheckBox" Text="{Loc 'ui-options-tts-enabled'}"/> <!-- Art-TTS -->

Не забудьте добавить соответствующие ключи в файлы локализации.

Also applies to: 19-19

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/Options/UI/Tabs/AudioTab.xaml` at line 11, В XAML убери жёстко
закодированные русские строки и замени их на локализуемые ресурсы, используя
шаблон {Loc 'KeyName'} для элементов SliderVolumeTts (Title) и соответствующего
чекбокса/элемента "Включить ТТС" (примерно на строке с именем, содержащим Tts
или EnableTts); добавь уникальные ключи (например Audio.VolumeTts и
Audio.EnableTts) в файлы локализации и обнови ресурсы перевода, чтобы сохранить
консистентность с остальными UI-элементами.

Content.Client/VoiceMask/VoiceMaskNameChangeWindow.xaml.cs-90-92 (1)

90-92: ⚠️ Potential issue | 🟡 Minor

Сбрасывайте выбор, если голос больше не доступен.

Если voice не найден в _voices, окно оставляет прошлый VoiceSelector как есть. При повторном открытии окна или смене списка голосов UI начнёт показывать устаревший выбор, который уже не соответствует серверному состоянию. Здесь нужен явный reset/fallback перед поиском.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/VoiceMask/VoiceMaskNameChangeWindow.xaml.cs` around lines 90 -
92, Reset the selector before attempting to re-select the voice so stale UI
state is cleared: set the VoiceSelector selection to an empty/fallback state
(e.g., SelectedIndex = -1 or clear selection) prior to calling
_voices.FindIndex(...), then if voiceIdx != -1 call
VoiceSelector.Select(voiceIdx); ensure you reference the existing
variables/methods (_voices, voiceIdx, VoiceSelector.Select) and add the explicit
reset when the voice is not found so the UI shows a cleared/fallback choice
instead of the previous selection.

Content.Client/Lobby/UI/HumanoidProfileEditor.xaml.cs-196-200 (1)

196-200: ⚠️ Potential issue | 🟡 Minor

Добавьте guard для UpdateTTSVoicesControls() или сделайте её безопасным no-op при выключенном TTS.

InitializeVoice() вызывается только под ArtCVars.TTSClientEnabled (строка 199), но UpdateTTSVoicesControls() вызывается без этого guard в SetProfile() (строка 300) и OnReset() (строка 380). Когда TTS отключён, _voiceList остаётся пустым, и UpdateTTSVoicesControls() очищает VoiceButton, но не заполняет его — результат: неконсистентное состояние UI (элемент управления существует, но всегда пуст). Либо добавьте защиту перед вызовами, либо убедитесь, что метод имеет ранний выход при отключённом TTS и не опирается на состояние из InitializeVoice().

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/Lobby/UI/HumanoidProfileEditor.xaml.cs` around lines 196 -
200, The UpdateTTSVoicesControls method can be called when TTS is disabled which
leaves _voiceList empty and yields an inconsistent UI; either add a guard before
calling it in SetProfile and OnReset (check
configurationManager.GetCVar(ArtCVars.TTSClientEnabled) or TTSContainer.Visible)
or change UpdateTTSVoicesControls to be a safe no-op when TTS is off (early
return if TTS disabled or if _voiceList is null/empty) so it does not rely on
InitializeVoice having run; update references to InitializeVoice, SetProfile,
OnReset, _voiceList, and VoiceButton accordingly.

Content.Client/_Art/TTS/HumanoidProfileEditor.TTS.cs-50-55 (1)

50-55: ⚠️ Potential issue | 🟡 Minor

Потенциальный IndexOutOfRangeException при пустом списке голосов.

Если _voiceList пуст, вызов _voiceList[firstVoiceChoiceId] (или _voiceList[0] после упрощения) приведёт к исключению.

🛡️ Предлагаемое исправление с проверкой

     var voiceChoiceId = _voiceList.FindIndex(x => x.ID == Profile.Voice);
-    if (!VoiceButton.TrySelectId(voiceChoiceId) &&
-        VoiceButton.TrySelectId(firstVoiceChoiceId))
+    if (!VoiceButton.TrySelectId(voiceChoiceId) &&
+        _voiceList.Count > 0 &&
+        VoiceButton.TrySelectId(0))
     {
-        SetVoice(_voiceList[firstVoiceChoiceId].ID);
+        SetVoice(_voiceList[0].ID);
     }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/HumanoidProfileEditor.TTS.cs` around lines 50 - 55,
The code in HumanoidProfileEditor.TTS.cs may throw IndexOutOfRangeException when
_voiceList is empty because it accesses _voiceList[firstVoiceChoiceId]; before
calling VoiceButton.TrySelectId and SetVoice, guard against an empty list (check
_voiceList.Count > 0 or _voiceList.Any()) and only call
VoiceButton.TrySelectId(…) or SetVoice(…) when there is at least one element;
update the block containing voiceChoiceId, firstVoiceChoiceId,
VoiceButton.TrySelectId and SetVoice to skip selection/setting (or use a safe
default) when _voiceList is empty.

Content.Server/_Art/TTS/TTSSystem.cs-66-72 (1)

66-72: ⚠️ Potential issue | 🟡 Minor

async void без await и отсутствие синхронизации для _ignoredRecipients.

1. Метод помечен как async void, но не содержит await — модификатор async не нужен.
2. Список _ignoredRecipients может модифицироваться из нескольких сетевых обработчиков без синхронизации.

♻️ Предлагаемое исправление

-private async void OnClientOptionTTS(ClientOptionTTSEvent ev, EntitySessionEventArgs args)
+private void OnClientOptionTTS(ClientOptionTTSEvent ev, EntitySessionEventArgs args)
 {
     if (ev.Enabled)
         _ignoredRecipients.Remove(args.SenderSession);
     else
         _ignoredRecipients.Add(args.SenderSession);
 }

Если требуется потокобезопасность, рассмотрите использование HashSet с блокировкой или ConcurrentDictionary.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/_Art/TTS/TTSSystem.cs` around lines 66 - 72, The handler
OnClientOptionTTS is marked async without any await and mutates the shared
_ignoredRecipients non-thread-safely; remove the async modifier and make the
method synchronous (change signature OnClientOptionTTS to a plain void handler)
and protect updates to _ignoredRecipients by either replacing the collection
with a concurrent structure (e.g., use a ConcurrentDictionary/ConcurrentHashSet
pattern keyed by args.SenderSession) or wrap Add/Remove in a lock using a
dedicated private readonly object (e.g., lock(_ignoredRecipientsLock) { ... });
ensure all code paths that modify or read _ignoredRecipients use the chosen
synchronization approach.

Content.Shared/_Art/TTS/TTSRadioPlayEvent.cs (1)

1-3: Неиспользуемые директивы using.

Импорты Content.Shared.Speech, Robust.Shared.Prototypes и Content.Shared.Inventory не используются в этом файле.

♻️ Предлагаемое исправление

-using Content.Shared.Speech;
-using Robust.Shared.Prototypes;
-using Content.Shared.Inventory;
-
 namespace Content.Shared._Art.TTS;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Shared/_Art/TTS/TTSRadioPlayEvent.cs` around lines 1 - 3, Удалите
неиспользуемые директивы using из файла TTSRadioPlayEvent.cs: уберите строки с
using Content.Shared.Speech, using Robust.Shared.Prototypes и using
Content.Shared.Inventory; оставьте только необходимые пространства имен и
сохраните объявление класса/структуры TTSRadioPlayEvent без изменений.

Content.Shared/_Art/TTS/TTSVoicePrototype.cs (1)

1-1: Неиспользуемый импорт Content.Shared.Humanoid.

♻️ Предлагаемое исправление

-using Content.Shared.Humanoid;
 using Robust.Shared.Prototypes;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Shared/_Art/TTS/TTSVoicePrototype.cs` at line 1, Удалите
неиспользуемую директиву использования `using Content.Shared.Humanoid;` из файла
— она не применяется в коде класса/файла `TTSVoicePrototype` и должна быть
удалена чтобы устранить предупреждение о неиспользуемом импорте.

Content.Shared/_Art/CVars/ArtCVars.cs (1)

41-45: Значение по умолчанию TTSVolume = 0f означает выключенный звук TTS.

Возможно, это намеренно, но пользователи могут не понять, почему TTS не работает. Рассмотрите значение 0.5f или 1.0f по умолчанию.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Shared/_Art/CVars/ArtCVars.cs` around lines 41 - 45, The default
TTSVolume CVar is set to 0f which mutes TTS by default; update the CVar default
in the TTSVolume declaration (the CVarDef.Create call for TTSVolume) to a
sensible audible default such as 0.5f (or 1.0f if you prefer full volume) so
users have working TTS out of the box while still allowing them to lower it via
the CVar.

Content.Server/_Art/TTS/TTSSystem.SSML.cs (1)

21-21: Опечатка в названии: PitchVerylow → PitchVeryLow.

Для согласованности с PascalCase рекомендуется переименовать в PitchVeryLow.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/_Art/TTS/TTSSystem.SSML.cs` at line 21, Переименуй
перечисление-элемент PitchVerylow в PitchVeryLow для соответствия PascalCase:
обнови определение enum (в файле TTSSystem.SSML.cs) и все места использования
этого значения (ссылки в коде, сравнения, сериализация/десериализация и тесты)
так, чтобы вместо PitchVerylow использовалось PitchVeryLow; не забудь также
обновить любые строковые представления или документацию, зависящие от имени.

Resources/_Art/TTS/tts-voices.yml (1)

1-37: Закомментированный код генератора следует удалить.

Закомментированные скрипты Emacs Lisp и Python не должны находиться в файле ресурсов. Переместите их в отдельный каталог /scripts или удалите.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Resources/_Art/TTS/tts-voices.yml` around lines 1 - 37, The file contains two
commented generator scripts (an Emacs Lisp defun f block and a Python
"#!/usr/bin/env python3" voice generator) embedded in
Resources/_Art/TTS/tts-voices.yml; remove these commented blocks from the YAML
and either delete them or move them into a proper scripts folder (e.g.,
/scripts) as standalone files (retain the Emacs Lisp function name "f" and the
Python generator that reads "voices.json" when moving so they can be located and
restored if needed), and ensure the YAML contains only resource data after the
change.

Content.Shared/VoiceMask/VoiceMaskComponent.cs (1)

18-22: Рассмотрите использование ProtoId<TTSVoicePrototype> вместо string.

Поле VoiceId объявлено как string, в то время как в HumanoidProfileComponent аналогичное поле Voice использует ProtoId<TTSVoicePrototype>. Для согласованности типов и типобезопасности при работе с прототипами рекомендуется использовать ProtoId<TTSVoicePrototype>.

♻️ Предлагаемое изменение

-    [DataField]
-    [ViewVariables(VVAccess.ReadWrite)]
-    public string VoiceId = TTSConfig.DefaultVoice;
+    [DataField]
+    [ViewVariables(VVAccess.ReadWrite)]
+    public ProtoId<TTSVoicePrototype> VoiceId = TTSConfig.DefaultVoice;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Shared/VoiceMask/VoiceMaskComponent.cs` around lines 18 - 22, Поле
VoiceId в VoiceMaskComponent объявлено как string; замените тип на
ProtoId<TTSVoicePrototype> для согласованности с
HumanoidProfileComponent::Voice, обновите инициализацию (вместо
TTSConfig.DefaultVoice присвойте new
ProtoId<TTSVoicePrototype>(TTSConfig.DefaultVoice) или эквивалентный
метод-конструктор), добавьте необходимый using/импорт для ProtoId и
TTSVoicePrototype и проверьте/скорректируйте все места, где читают/пишут VoiceId
(например сравнения или сериализация), чтобы работать с
ProtoId<TTSVoicePrototype> вместо string.

Content.Server/Radio/EntitySystems/HeadsetSystem.cs (1)

113-123: Дублирование вызова Transform(uid).ParentUid.

Переменная parent уже вычислена на строке 105. На строке 120 повторно вызывается Transform(uid).ParentUid. Рекомендуется использовать существующую переменную parent.

♻️ Предлагаемое исправление

         // Art-TTS Start
         if (TryComp(parent, out ActorComponent? actor))
         {
             _netMan.ServerSendMessage(args.ChatMsg, actor.PlayerSession.Channel);
             if (args.Voice is string voice)
             {
                 var ev = new TTSRadioPlayEvent(args.Message, voice, GetNetEntity(uid), GetNetEntity(args.MessageSource));
-                RaiseLocalEvent(Transform(uid).ParentUid, ev);
+                RaiseLocalEvent(parent, ev);
             }
         }
         // Art-TTS End

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/Radio/EntitySystems/HeadsetSystem.cs` around lines 113 - 123,
The code duplicates Transform(uid).ParentUid even though parent was already
computed; inside the TryComp(actor) block where you create the TTSRadioPlayEvent
and call RaiseLocalEvent, replace the repeated Transform(uid).ParentUid call
with the existing parent variable (computed earlier), keeping the rest of the
logic (creating TTSRadioPlayEvent(args.Message, voice, GetNetEntity(uid),
GetNetEntity(args.MessageSource)) and calling RaiseLocalEvent(parent, ev))
intact so you no longer recompute the transform.

Content.Client/_Art/TTS/ContentAudioSystem.cs (1)

1-8: Документирование "магического числа" TtsMultiplier

Файл корректно расположен в Content.Client/_Art/TTS/, но использует namespace Content.Client.Audio — это правильный паттерн для partial-классов, так как основное определение ContentAudioSystem находится в Content.Client/Audio/.

Рекомендуется добавить комментарий для объяснения значения константы TtsMultiplier = 3f:

 using Content.Shared.Audio;
 
 namespace Content.Client.Audio;
 
 public sealed partial class ContentAudioSystem : SharedContentAudioSystem
 {
+    /// <summary>
+    /// Множитель громкости для TTS-аудио для нормализации воспринимаемой громкости.
+    /// </summary>
     public const float TtsMultiplier = 3f;
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/ContentAudioSystem.cs` around lines 1 - 8, The
constant TtsMultiplier = 3f in the partial class ContentAudioSystem is a magic
number and should be documented; add a short explanatory comment or XML doc
summary above the TtsMultiplier declaration (referencing ContentAudioSystem and
TtsMultiplier) that describes what the multiplier means, its units/scale, why
the value 3f was chosen, and any implications for audio timing/volume so future
maintainers understand its purpose.

Content.Client/_Art/TTS/TTSSystem.cs (3)

45-50: Рассмотрите использование private вместо internal для _toDelete.

Поле _toDelete помечено как internal, но используется только внутри этого класса. Рекомендуется использовать private для лучшей инкапсуляции.

♻️ Предлагаемое исправление

-internal List<NetEntity> _toDelete = new();
+private readonly List<NetEntity> _toDelete = new();

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/TTSSystem.cs` around lines 45 - 50, The field
_toDelete is declared internal but is only used inside this class; change its
accessibility to private by renaming its declaration from internal
List<NetEntity> _toDelete = new(); to private List<NetEntity> _toDelete = new();
and run a quick search for external references to _toDelete to ensure no
external usages break (if any exist, either keep internal or update those call
sites).

---

175-185: Рассмотрите вынесение магического числа 3.0f в константу.

Множитель 3.0f в расчёте громкости не имеет документации. Рекомендуется вынести его в именованную константу для ясности.

+/// <summary>
+/// Volume multiplier for TTS audio.
+/// </summary>
+private const float VolumeMultiplier = 3.0f;
+
 private float AdjustVolume(bool isWhisper)
 {
-    var volume = MinimalVolume + SharedAudioSystem.GainToVolume(_volume * 3.0f);
+    var volume = MinimalVolume + SharedAudioSystem.GainToVolume(_volume * VolumeMultiplier);

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/TTSSystem.cs` around lines 175 - 185, В методе
AdjustVolume используется "магическое" число 3.0f при вычислении громкости (var
volume = MinimalVolume + SharedAudioSystem.GainToVolume(_volume * 3.0f));
вынесите этот множитель в именованную константу (например VolumeGainMultiplier
или DefaultGainScale) в области класса рядом с MinimalVolume/WhisperFade,
замените 3.0f на константу и добавьте короткий комментарий о назначении
константы; проверьте, что AdjustVolume, _volume, SharedAudioSystem.GainToVolume
и WhisperFade по-прежнему корректно работают с новой константой.

---

158-158: Рекомендуется использовать using для MemoryStream.

Хотя MemoryStream не содержит неуправляемых ресурсов, использование using является хорошей практикой для IDisposable объектов.

♻️ Предлагаемое исправление

-var audioStream = _audioLoader.LoadAudioWav(new MemoryStream(ev.Data));
+using var memoryStream = new MemoryStream(ev.Data);
+var audioStream = _audioLoader.LoadAudioWav(memoryStream);

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/TTSSystem.cs` at line 158, Объявление MemoryStream
при вызове _audioLoader.LoadAudioWav(new MemoryStream(ev.Data)) не обёрнуто в
using; оберните создание MemoryStream в using (или используйте using var) чтобы
гарантировать Dispose после использования; местоположения для правки: вызов в
методе в классе TTSSystem.cs где создаётся переменная audioStream и вызывается
метод LoadAudioWav на _audioLoader.

Content.Client/_Art/TTS/HumanoidProfileEditor.TTS.cs (1)

38-48: Упростите логику выбора первого голоса.

Переменная firstVoiceChoiceId инициализирована значением 1, но условие if (firstVoiceChoiceId == 1) сразу устанавливает её в i (что равно 0 на первой итерации). Это эквивалентно простому использованию 0 как индекса по умолчанию.

♻️ Предлагаемое упрощение

     VoiceButton.Clear();

-    var firstVoiceChoiceId = 1;
     for (var i = 0; i < _voiceList.Count; i++)
     {
         var voice = _voiceList[i];

         var name = Loc.GetString(voice.Name);
         VoiceButton.AddItem(name, i);
-
-        if (firstVoiceChoiceId == 1)
-            firstVoiceChoiceId = i;
     }

     var voiceChoiceId = _voiceList.FindIndex(x => x.ID == Profile.Voice);
     if (!VoiceButton.TrySelectId(voiceChoiceId) &&
-        VoiceButton.TrySelectId(firstVoiceChoiceId))
+        VoiceButton.TrySelectId(0))
     {
-        SetVoice(_voiceList[firstVoiceChoiceId].ID);
+        SetVoice(_voiceList[0].ID);
     }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Client/_Art/TTS/HumanoidProfileEditor.TTS.cs` around lines 38 - 48,
The logic around firstVoiceChoiceId is redundant: it's initialized to 1 then
immediately set to i on the first loop iteration; simplify by initializing
firstVoiceChoiceId to 0 (or remove it and use 0 directly) and drop the if check
inside the loop. Update the loop over _voiceList (where
VoiceButton.AddItem(name, i) is called) to rely on a default index of 0 for the
first voice choice and remove the conditional that references
firstVoiceChoiceId.

Content.Server/_Art/TTS/TTSSystem.cs (1)

74-89: Рекомендуется добавить обработку исключений в async void методах.

Методы async void не позволяют отслеживать исключения — они могут привести к необработанным ошибкам. Рекомендуется обернуть тело метода в try-catch с логированием.

♻️ Пример обработки исключений

 private async void OnRequestPreviewTTS(RequestPreviewTTSEvent ev, EntitySessionEventArgs args)
 {
+    try
+    {
         if (!_isEnabled ||
             !_prototypeManager.TryIndex<TTSVoicePrototype>(ev.VoiceId, out var protoVoice))
             return;

         if (HandleRateLimit(args.SenderSession) != RateLimitStatus.Allowed)
             return;

         var previewText = _rng.Pick(_sampleText);
         var soundData = await GenerateTTS(previewText, protoVoice.Speaker);
         if (soundData is null)
             return;

         RaiseNetworkEvent(new PlayTTSEvent(soundData, null), Filter.SinglePlayer(args.SenderSession));
+    }
+    catch (Exception ex)
+    {
+        Log.Error(ex, "Error processing TTS preview request");
+    }
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@Content.Server/_Art/TTS/TTSSystem.cs` around lines 74 - 89,
OnRequestPreviewTTS is an async void handler so exceptions can escape; wrap the
method body in a try/catch that catches Exception (or a more specific set) and
logs the error (including context like ev.VoiceId, args.SenderSession) before
returning, ensuring failures from GenerateTTS, _prototypeManager.TryIndex, or
RaiseNetworkEvent are logged; keep existing early returns (RateLimitStatus
check) but put the remaining logic (previewText selection, await GenerateTTS,
and RaiseNetworkEvent) inside the try block and log the exception in the catch
(include method name OnRequestPreviewTTS and relevant identifiers).